Skip to content

Add TiCI dashboard support to platform-monitoring/operator/init.sh#994

Merged
ti-chi-bot[bot] merged 3 commits intomasterfrom
copilot/sync-init-script-changes
Feb 13, 2026
Merged

Add TiCI dashboard support to platform-monitoring/operator/init.sh#994
ti-chi-bot[bot] merged 3 commits intomasterfrom
copilot/sync-init-script-changes

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026 •
edited
Loading

The platform-monitoring operator init script was missing support for TiCI dashboards.

Changes

  • Add TiCI dashboard support: Added configuration for three TiCI dashboard types:
    • TiCI Meta dashboard
    • TiCI Worker dashboard
    • TiCI Reader dashboard

Each dashboard is copied from /tmp and processed to replace the test cluster name with the standard "Cluster" naming convention used in the operator context.

The changes are minimal and focused, adding only the TiCI dashboard section without modifying any existing functionality.

Original prompt

The user wants to apply recent changes from cmd/init.sh to platform-monitoring/operator/init.sh.

However, I need to verify if cmd/init.sh exists or if the user meant another file (possibly the one they provided in the initial prompt or a file in a different branch/PR).

Task:

  1. Locate cmd/init.sh and platform-monitoring/operator/init.sh.
  2. Sync the changes from cmd/init.sh (or the intended source) to platform-monitoring/operator/init.sh.
  3. If cmd/init.sh doesn't exist, search for where the "recent changes" might be (e.g. in a specific PR or another file path).

Based on the conversation, the user provided the content of a file which they might be referring to.

Repo: pingcap/monitoring
Base branch: master

This pull request was created from Copilot chat.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@wuhuizuo
Sync changes from cmd/init.sh to platform-monitoring/operator/init.sh
5cb2e71 
Co-authored-by: wuhuizuo <2574558+wuhuizuo@users.noreply.github.com>
@ti-chi-bot ti-chi-bot bot added size/M and removed size/XS labels Feb 12, 2026
Copilot AI changed the title [WIP] Sync changes from cmd/init.sh to platform-monitoring/operator/init.sh Sync platform-monitoring/operator/init.sh with cmd/init.sh Feb 12, 2026
Copilot AI requested a review from wuhuizuo February 12, 2026 12:45
@wuhuizuo
Copy link
Contributor

wuhuizuo commented Feb 12, 2026

@copilot only apply the change of TiCI dashboard, do not change others.

@wuhuizuo
Add only TiCI dashboard support to platform-monitoring/operator/init.sh
a486c89 
Co-authored-by: wuhuizuo <2574558+wuhuizuo@users.noreply.github.com>
@ti-chi-bot ti-chi-bot bot added size/XS and removed size/M labels Feb 12, 2026
Copy link
Contributor Author

Copilot AI commented Feb 12, 2026

@copilot only apply the change of TiCI dashboard, do not change others.

Done. Reverted all changes except the TiCI dashboard section. Only added support for TiCI Meta, Worker, and Reader dashboards as requested (commit a486c89).

Copilot AI changed the title Sync platform-monitoring/operator/init.sh with cmd/init.sh Add TiCI dashboard support to platform-monitoring/operator/init.sh Feb 12, 2026
@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 12, 2026

@3pointer: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 12, 2026

@wuhuizuo: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@wuhuizuo wuhuizuo marked this pull request as ready for review February 12, 2026 14:21
@ti-chi-bot ti-chi-bot bot requested a review from liubog2008 February 12, 2026 14:21
@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 3pointer, csuzhangxc, wuhuizuo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added the lgtm label Feb 13, 2026
@ti-chi-bot
Copy link

ti-chi-bot bot commented Feb 13, 2026

[LGTM Timeline notifier]

Timeline:

  • 2026-02-13 03:19:17.125870214 +0000 UTC m=+161310.205860909: ☑️ agreed by csuzhangxc.

@ti-chi-bot ti-chi-bot bot added the approved label Feb 13, 2026
@ti-chi-bot ti-chi-bot bot merged commit 955024f into master Feb 13, 2026
2 checks passed
@ti-chi-bot ti-chi-bot bot deleted the copilot/sync-init-script-changes branch February 13, 2026 03:23
This was referenced Feb 13, 2026
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@csuzhangxc csuzhangxc csuzhangxc approved these changes

@3pointer 3pointer 3pointer approved these changes

@liubog2008 liubog2008 Awaiting requested review from liubog2008

+1 more reviewer

@wuhuizuo wuhuizuo wuhuizuo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@wuhuizuo wuhuizuo

Copilot code review Copilot

Labels

approved lgtm size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@wuhuizuo @csuzhangxc @3pointer